Skip to content

Rebuild for 0.8.14. #953

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 23, 2018
Merged

Rebuild for 0.8.14. #953

merged 2 commits into from
Feb 23, 2018

Conversation

rhelgeby
Copy link

@rhelgeby rhelgeby commented Feb 19, 2018

Description

Rebuild support/0.x so that the merge of pull request #906 is included in the build.

Even though it may not be ready for release, this allows other people to reference and test the support/0.x branch directly, using GitHub references in NPM's package.json.

Fixes Related issues

Checklist

  • I have read and understand the CONTRIBUTIONS.md file
  • I have searched for and linked related issues
  • I have created test cases to ensure quick resolution of the PR is easier
  • I am NOT targeting main branch
  • I did NOT include the dist folder in my PR

@json-schema-form/angular-schema-form-lead

@rhelgeby
Copy link
Author

rhelgeby commented Feb 19, 2018

The build fails with a version conflict on jQuery. This also happened in build 246465021.

I don't know if the support/0.x branch would build before, but when I built it I resolved it with jQuery 2.x. Maybe this resolution should be marked in bower.json?

@Anthropic
Copy link
Member

@rhelgeby by all means update the bower resolution in this PR and I will accept it for sure, thanks for taking the time to help, I appreciate it :)

Resolving as version 2.x because that's the version angular-schema-form depends on. If there's an issue with angular-ui-sortable, it the resolution should probably be increased to 3.x.

This was the original version conflict:

Unable to find a suitable version for jquery, please choose one by typing one of the numbers below:
    1) jquery#~2.1.1 which resolved to 2.1.4 and is required by angular-schema-form
    2) jquery#>=1.6 which resolved to 2.1.4 and is required by jquery-ui#1.12.1
    3) jquery#>=1.7 which resolved to 2.1.4 and is required by pickadate#3.5.6
    4) jquery#>=1.7.2 which resolved to 2.1.4 and is required by spectrum#1.7.0
    5) jquery#>=3.1.x which resolved to 3.3.1 and is required by angular-ui-sortable#0.19.0
@Anthropic Anthropic merged commit adef004 into json-schema-form:support/0.x Feb 23, 2018
@Anthropic
Copy link
Member

Thanks @rhelgeby let me know after a little while if you think it is stable enough for a release :)

@iCodr8
Copy link

iCodr8 commented Jul 24, 2018

What about a release?

@rhelgeby
Copy link
Author

rhelgeby commented Jul 24, 2018

We use this version in production already and I suppose it's considered stable now.

I've forked and built my own version and reference my branch for this pull request directly: "angular-schema-form": "github:rhelgeby/angular-schema-form#build"

Warning: You can reference my fork as a temporary workaround. But don't rely on me to keep this branch or fork forever. I won't give notices of changes. It's better to have an official release and package in the NPM repository.

@iCodr8
Copy link

iCodr8 commented Jul 25, 2018

I have tested it and it works wonderful! Thank you!

The only problem was the way to use it. I had to use this variant:

"angular-schema-form": "git@github.com:rhelgeby/angular-schema-form.git#build"

This version should be in this repository as version 0.8.14.

@rhelgeby
Copy link
Author

Great. Even though I use this in production, again, remember that you use my fork on your own risk. I can't guarantee anything. If you want to ensure stability you could fork my clone and reference your own repository.

@Anthropic Could you make an official release for 0.8.14 when time permits? It's best if we all switch to an official version.

@Anthropic
Copy link
Member

@rhelgeby I will, yes, glad it works for you all I will get onto it shortly, I'm recovering from surgery so there's been a slight delay in me getting to things as I am heavily medicated during recovery.

@Anthropic
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants